docs: Use serverpod start in provider docs#591
Conversation
…rpod start' instead of 'dart run bin/main.dart --apply-migrations'
…size starting the server
…s to focus on starting the server
|
It should say "Part of", not "Closes" in the PR description, right? |
|
Hmm it could be that also but this touches all the places we have that in the next doc except migrations |
| - [ ] Add `FirebaseIdpConfigFromPasswords()` to `identityProviderBuilders` in `server.dart`. | ||
| - [ ] Create a `FirebaseIdpEndpoint` file in `lib/src/auth/` extending `FirebaseIdpBaseEndpoint`. | ||
| - [ ] Run **`serverpod generate`**, then **`serverpod create-migration`**, then apply migrations with `dart run bin/main.dart --apply-migrations`. | ||
| - [ ] Run **`serverpod generate`**, then **`serverpod create-migration`**, then start the server with `serverpod start`. |
There was a problem hiding this comment.
Following up on @Swiftaxe earlier note: Google, Apple, and GitHub were all fully converted to serverpod start, but Firebase still has the explicit serverpod generate + serverpod create-migration chain. If serverpod start handles those (as the rest of the PR assumes), Firebase should match.
| - [ ] Run **`serverpod generate`**, then **`serverpod create-migration`**, then start the server with `serverpod start`. | |
| - [ ] Start the server with `serverpod start`. |
| - [ ] Add `GoogleIdpConfigFromPasswords()` to `identityProviderBuilders` in `server.dart`. | ||
| - [ ] Create a `GoogleIdpEndpoint` file in `lib/src/auth/`. | ||
| - [ ] Run `serverpod generate`, then `serverpod create-migration`, then apply migrations using `--apply-migrations`. | ||
| - [ ] Started the server with `serverpod start`. |
There was a problem hiding this comment.
Tense mix here: the new Started the server with serverpod start. is past tense next to imperative siblings (Add, Create). GitHub's checklist is already all past tense, which reads more naturally for a "what have you done" checkbox list. Two options:
- Convert the rest of the list to past tense (matches GitHub).
- Keep imperative on the new line for consistency with the existing items:
Start the server with serverpod start.
Either resolves the inconsistency.
| * [ ] Call **`pod.configureAppleIdpRoutes(...)`** on the server before the pod starts. | ||
| * [ ] Create an `AppleIdpEndpoint` file in `lib/src/auth/`. | ||
| * [ ] Run **`serverpod generate`**, then apply migrations using `--apply-migrations`. | ||
| * [ ] Started the server with `serverpod start`. |
There was a problem hiding this comment.
Same tense mix as in the Google file: the new Started the server with serverpod start. is past tense next to imperative siblings (Add, Call, Create). Either convert the whole list to past tense (matches GitHub's checklist), or keep imperative on this line for consistency with the existing items: Start the server with serverpod start.
3 participants
Summary
Replaces
dart run bin/main.dart --apply-migrationswithserverpod startacross the Google, Apple, GitHub, and Firebase provider docs (setup + troubleshooting). The explicit forms stay documented on the migrations reference and the--role maintenance/--apply-repair-migrationcallsites.Part of #565